xen/sched: fix race in RTDS scheduler
authorJuergen Gross <jgross@suse.com>
Mon, 31 Oct 2022 12:22:54 +0000 (13:22 +0100)
committerJan Beulich <jbeulich@suse.com>
Mon, 31 Oct 2022 12:22:54 +0000 (13:22 +0100)
commit481465f35da1bcec0b2a4dfd6fc51d86cac28547
tree8514ee5e520e4736023f86b309e84ac1eec5f7d3
parent54f8ed80c8308e65c3f57ae6cbd130f43f5ecbbd
xen/sched: fix race in RTDS scheduler

When a domain gets paused the unit runnable state can change to "not
runnable" without the scheduling lock being involved. This means that
a specific scheduler isn't involved in this change of runnable state.

In the RTDS scheduler this can result in an inconsistency in case a
unit is losing its "runnable" capability while the RTDS scheduler's
scheduling function is active. RTDS will remove the unit from the run
queue, but doesn't do so for the replenish queue, leading to hitting
an ASSERT() in replq_insert() later when the domain is unpaused again.

Fix that by removing the unit from the replenish queue as well in this
case.

Fixes: 7c7b407e7772 ("xen/sched: introduce unit_runnable_state()")
Signed-off-by: Juergen Gross <jgross@suse.com>
Acked-by: Dario Faggioli <dfaggioli@suse.com>
master commit: 73c62927f64ecb48f27d06176befdf76b879f340
master date: 2022-10-21 12:32:23 +0200
xen/common/sched/rt.c